Skip to content

Conversation

@ivokwee
Copy link
Member

@ivokwee ivokwee commented Jan 16, 2026

Summary

  • Adds AI-related functions to support the AI Copilot module in omicsplayground
  • Removes debug logging from ai.create_report()

Dependencies

⚠️ Note: This PR is required for the corresponding omicsplayground PR #1683.

Test plan

  • Verify AI functions work correctly with the omicsplayground AI Copilot module
  • Confirm existing functionality remains unaffected

🤖 Generated with Claude Code

ivokwee and others added 2 commits January 15, 2026 17:38
Clean up dbg() statements that were used during development.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@ivokwee ivokwee requested review from ESCRI11 and zitoa January 16, 2026 15:15
@ESCRI11
Copy link
Contributor

ESCRI11 commented Jan 16, 2026

im trying to update the NAMESPACE to include the new exports and I run into this error

rror in match(x, table, nomatch = 0L) : 
  'S4SXP': should not happen - please report
Calls: <Anonymous> ... roclet_process.roclet_namespace -> warn_missing_s3_exports -> %in%
Execution halted

is my laptop problem or something is wrong?

@ESCRI11
Copy link
Contributor

ESCRI11 commented Jan 16, 2026

Ok so it seems its just my issue, fantastic

@ESCRI11
Copy link
Contributor

ESCRI11 commented Jan 19, 2026

after latest commit I cant install because ai.get_ollama_models function call is on line 1 of ai-llm.R file and defined later

@zitoa
Copy link
Contributor

zitoa commented Jan 19, 2026

PR not ready to be approved. Error in ai.get_ollama_models() :
could not find function "ai.get_ollama_models"
Error: unable to load R code in package ‘playbase’

@ivokwee
Copy link
Member Author

ivokwee commented Jan 19, 2026

oh yes. I need to push that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants